Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added GNB Lv40, Lv50, L60, L70 support #362

Merged
merged 4 commits into from
Jun 5, 2024
Merged

Conversation

Akechi-kun
Copy link
Contributor

There were certain bugs within the code that were causing Autorotation to brick in plenty of instances regarding Lv40-Lv72.

This is a fix for that.

I added support for lower levels, preferably Lv40-70

  • Lv40: Added logic to make sure ST and AOE combo works effectively the same as Lv50 combo.
  • Lv50: Added logic to ensure the appropriate skills are being used for the correct rotation. No Mercy was not being procced on ST or AOE & when you used it manually, it would get stuck. Added a couple !state checks to ensure no funny business with locked skills due to level sync. (fuck off Gnashing Fang)
  • Lv60: Fixed AOE combo issue; Kept getting stuck on Fated Circle, so added a !state check for appropriate skills to ensure Burst Strike (or Gnashing Fang) is used, regardless of ST or AOE.
  • Lv70: basically same shit as Lv60.

Akechi-kun and others added 3 commits June 3, 2024 22:51
There were certain bugs within the code that were causing Autorotation to brick in plenty of instances regarding Lv40-Lv72. 

This is a fix for that. 

I added support for lower levels, preferably Lv40-70
- Lv40: Added logic to make sure ST and AOE combo works effectively the same as Lv50 combo.
- Lv50: Added logic to ensure the appropriate skills are being used for the correct rotation. No Mercy was not being procced on ST or AOE & when you used it manually, it would get stuck. Added a couple !state checks to ensure no funny business with locked skills due to level sync. (fuck off Gnashing Fang)
- Lv60: Fixed AOE combo issue; Kept getting stuck on Fated Circle, so added a !state check for appropriate skills to ensure Burst Strike (or Gnashing Fang) is used, regardless of ST or AOE. 
- Lv70: basically same shit as Lv60.
This reverts commit c0bc567.
@Akechi-kun Akechi-kun closed this Jun 4, 2024
@Akechi-kun Akechi-kun reopened this Jun 4, 2024
@Akechi-kun
Copy link
Contributor Author

I'm not sure why these keep failing the build checks, as I am only within GNBRotation.cs, however if there's something I am missing within the framework please lmk.

@CarnifexOptimus
Copy link
Contributor

I'm not sure why these keep failing the build checks, as I am only within GNBRotation.cs, however if there's something I am missing within the framework please lmk.

you can click on the details of the failed check to see which changes need to be made

@Akechi-kun Akechi-kun changed the title Added Lv40, Lv50, L60, L70 support Added GNB Lv40, Lv50, L60, L70 support Jun 5, 2024
@awgil awgil merged commit 7e19aa1 into awgil:master Jun 5, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants